Skip to content

Conversation

@FullGas1
Copy link

@FullGas1 FullGas1 commented Jul 17, 2024

1 - when param oType is not sended to dbUpdate(event, oType, origGroupName), oType need to be defined in dbUpdate fct
2- function writeDBTables(newEntry) need newEntry.data but mist.forceAddToDB(object) send datas directly in newEntry

mrSkortch and others added 4 commits July 2, 2024 16:33
MODIFIED: dbUpdate to check getPlayerName on the object to add dynamicSlot value
MODIFIED: dbUpdate to save skill to client if it is from a dynamic slot
MODIFIED: groupSpawned event to check playerName and MEunitsByName to add the unit to tempSpawnedGroups
MODIFIED: writeDBTables to check if dynamicSlot is a passed value. If present will update mist.DBs.humansByName and mist.DBs.humansById for any player spawned in a dynamic slot
FIXED: mist.random to just call math.random if the highNum is larger than 50.
2 fixes detailled in Issues

Thank's
@JustWantToReadDiscussion

I'm not having any luck getting this to run. It throws an error at line 2162: '' expected near 'end'

@FullGas1
Copy link
Author

Hello,
sorry I'm not used to collaborating on GitHub, and I think I worked a little too quickly.

As a result, I hadn't seen that in your version 4_5_127 you had already dealt with dynamic spwans. For my proposal for managing dynamic spwans to be more readable and not mixed up with your version, I should have started from stable 4_5_126.

The easiest way to not complicate everything is to only keep the 2 small corrections that I propose in commit n° fad41b6, and which seem useful to me.

If you are interested in my version of managing dynamic spwans, I can provide it to you on request. It consists of only 1 main function:
> mist.updateMISTDBsDynSpwans(params, t)
and 4 utility functions:
> function mist.utils.mergeTables()
> function mist.utils.isEmptyTable(myTable)
> function mist.utils.isPairsTable(t)
> function mist.utils.isIpairsTable(t)

In any case, thank you for what you offer us via MIST.

Kind regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants